-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
allow custom module positions #3231
Conversation
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. @@ Coverage Diff @@
## develop #3231 +/- ##
===========================================
+ Coverage 25.19% 25.30% +0.10%
===========================================
Files 54 54
Lines 11933 11943 +10
===========================================
+ Hits 3007 3022 +15
+ Misses 8926 8921 -5
|
and do you think about change module position when MM² is running ? sample using with an pages module
Actually they have to use multi-instance of a module with |
there is a module that does this now in 3rd party list |
I already forgot this PR ...
not before your comment ... This PR was only meant for discussing custom module positions, the implementation is a little ugly, not sure if this should go live ... |
I think it would be nice if we could give the users more flexibility with this option. I might prefer a more elegant solution. But I can't think of one at the moment 😅 |
the problem is that I tell myself... I would have tried to code it but I'm recoding on one of my modules (and it's giving me a headache...) When I've done it...why don't I start coding it. (this might be interesting) Note: sometimes it would be good to have a TODO file :p |
I think we should do this with issues, there is already #3212 but I can create a new one for this feature (and assign it to you if you want). |
I submitted #3518 which reads index.html to get the defined regions for this run |
this is superceded by #3518 |
I'm not sure if this should be merged but I tested this because of these discussions:
With this PR you can
config.js
, key ismodulePositions
css
directory, the name of this file is specified inconfig.js
, key ismodulePositionsCss
So let me know whether we should offer this or not (no problem for me if not). This PR could be a basis for that.